Skip to content

[DOCS] Improve clarity of DBM Azure Postgres setup page#35917

Open
michaelperalta wants to merge 1 commit intomasterfrom
mike.peralta/dbm-postgres-azure-clarity
Open

[DOCS] Improve clarity of DBM Azure Postgres setup page#35917
michaelperalta wants to merge 1 commit intomasterfrom
mike.peralta/dbm-postgres-azure-clarity

Conversation

@michaelperalta
Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

Clarity improvements to the Database Monitoring setup page for Azure Postgres.

  • Split the parameters table into Required and Optional sections
  • Added explicit ### Create the explain plan function heading (was orphaned after version tabs)
  • Fixed the Postgres 9.6 "Verify" tab — it was identical to the ≥10 tab; now correctly uses function forms required for 9.6
  • Renamed verification sections for consistency: "Verify database permissions" and "Verify Agent setup"
  • Removed "Please" from the Entra ID authentication note
  • Simplified the "Grant the Agent access" intro paragraph
  • Fixed passive voice
  • Added missing period to Troubleshooting sentence

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

- Split config parameters table into Required/Optional sections
- Add explicit heading for explain plan function
- Fix 9.6 Verify tab to use function form (was identical to ≥10 tab)
- Rename sections: "Verify database permissions" and "Verify Agent setup"
- Remove "Please" from Entra ID note
- Simplify Grant intro paragraph; fix passive voice
- Add missing period to Troubleshooting sentence

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@michaelperalta michaelperalta requested review from a team as code owners April 10, 2026 17:28
@github-actions
Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@git-thuerk-done
Copy link
Copy Markdown
Contributor

git-thuerk-done commented Apr 10, 2026

Editorial review tracked in DOCS-13973 (batch review with PRs #35912, #35913, #35914, #35915, #35916, #35918).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants